Merge branch 'wip/chergert/fix-bgr8-uploads' into 'master'

gl: handle B8G8R8 texture data when uploading

See merge request GNOME/gtk!3171

gl: handle B8G8R8 texture data when uploading

We do not need to translate this on the CPU when we can instead push it
to the GPU in the same format and allow it to swizzle.

This fixes a huge number of memory allocations found while uploading the
GTK animation in widget-factory.

Merge branch 'wip/chergert/macos-perf-fixes' into 'master'

macos: various OpenGL performance fixes

See merge request GNOME/gtk!3170

macos: improve use of swap rectangles with OpenGL

macos: only flush when not attached

This makes it so we only flush the context for the NSView, not the context
that is the center of our center of the GL context spokes.

macos: mark region as unlikely

gsk: make gsk_render_node_get_node_type() use const

There isn't any state to modify in the type so we can use const here.
Doing so allows some of the renderer code to use const across a
number of functions so that repeated calls are elided if inlined.

gsk: make render node getters pure

These do not do modify anything so they can be marked as pure to
potentially ellide calls. Since they do dereference, I do not believe
we can make them const although that is unclear since we could technically
just return a pointer + offset. Therefore it *might* be possible to also
make these G_GNUC_CONST.

gsk: make render node getters const

This also removes the return if fail macros from these as a good portion
of them didn't have them anyway. I think it's fair to say that access to
these incorrectly is a programmer error.

It significantly reduces the amount of code generated into generally a
movss,ret.

profiler: allow using gdk profiler functions without braces

Just compiling these out means you have to write code slightly differently
so that you don't end up with "if ();" afterwards.

This adds a "do {} while (0)" so that we're still semantically a statement
but will also compile out.

Merge branch 'wip/exalm/headerbar' into 'master'

headerbar: Don't use gtk_widget_unparent() with GtkBox

See merge request GNOME/gtk!3169

headerbar: Null-check boxes when removing window controls

Prevent a crash when show-title-buttons value changes to FALSE during
destruction.

headerbar: Don't use gtk_widget_unparent() with GtkBox

gtk_widget_unparent() is for widget implementations, it's not supposed to
be used from outside.

Merge branch 'printing-temporary-queues-gtk4' into 'master'

printing: Create temporary queues for Avahi printers

See merge request GNOME/gtk!3160

Merge branch 'focus' into 'master'

listbox: Fix handling of non-focusable rows

Closes #3633

See merge request GNOME/gtk!3152

Merge branch 'fix-x11-drop' into 'master'

x11: More dnd fixes

Closes #3654

See merge request GNOME/gtk!3166

x11: More dnd fixes

Whenever we communicate targets, we need to the union, otherwise
we don't tell the other side about our serialization. This makes
drops of images from gtk4-icon-browser to gimp and libreoffice
succeed in transferring data.

Fixes: #3654

Merge branch 'fix-x11-drop' into 'master'

Fix x11 drop

Closes #3652

See merge request GNOME/gtk!3165

x11: Pass mimetypes when creating output stream

When creating the output stream for a drop, we must
pass the mimetypes we support, otherwise the picking
of the right handler does not work.

Fixes: #3652

x11: Fix a memory leak

We ref the content formats here, so we better unref it
when done.

x11: Some code cleanups

Fix some glaring misindentation.

Merge branch 'matthiasc/for-master' into 'master'

Implement drop-shadow css filter

See merge request GNOME/gtk!3163

Add tests for the css filter property

Test that we can parse filters and that we
don't accept invalid filters.

cssfiltervalue: Reject invalid filters

The Filter Effects Spec doesn't allow negative values
for most of the filter parameters.

cssfiltervalue: Fix blur filter interpretation

According to https://www.w3.org/TR/filter-effects-1/,
the length passed to blur() is the standard deviation,
and according to https://www.w3.org/TR/css-backgrounds-3/#shadow-blur
the blur radius is twice the standard deviation.

cssfiltervalue: Implement drop-shadows

We have all the pieces, so this is surprisingly easy.

cssshadowvalue: Add a 'filter mode'

Shadow values created by gtk_css_shadow_value_new_filter or
gtk_css_shadow_value_parse_filter interpret their radius value
as standard deviation. Add a flag for this mode, and use it
where necessary.

cssshadowvalue: Add a parsing function

Add a variant of the parse function that parses
just a single (non-box) shadow, as required for
the drop-shadow filter.

cssshadowvalue: Add gtk_css_shadow_value_pop_snapshot

This is the counterpart ot gtk_css_shadow_value_push_snapshot.

To make this easy, move the determination whether we need a
shadow out of the push function and save it.

cssshadowvalue: Drop the underscore

Rename _gtk_css_shadow_value_parse to drop the underscore.
It was the only underscore-prefixed function in this file.

Updated Czech translation

Add tests for the css filter property

Test that we can parse filters.

css: Implement the drop-shadow filter

We have all the pieces, so this is surprisingly easy.

cssshadowvalue: Add a parsing function

Add a variant of the parse function that parses
just a single (non-box) shadow, as required for
the drop-shadow filter.

Merge branch 'module-ext' into 'master'

meson: use correct module file extension on macOS

Closes #3645

See merge request GNOME/gtk!3162

meson: use correct module file extension on macOS

GModule requires the .so file extension on macOS for historic reasons.
However Meson defaults to .dylib for modules, so we need to override
it to get the correct extension.

Fixes #3645.

printing: Show all Avahi advertised printers

This commit unsubscribes CUPS backend from a DBus
signal in idle when listening for new items on Avahi.

Since GDBus emits gathered signals in idle while
checking whether the signal has been unsubscribed
it could happen that a signal was not processed
because it was removed from hash table of
subscribed signals.
This caused the situation where printers advertised
on Avahi were not listed in CUPS backend sometimes.

We need those signals since this happens when switching
from a general subscription which listens to signals
for all Avahi services to a specific one which listens
to just _ipp._tcp and _ipps._tcp (chicken and egg problem).

printing: Create temporary queues for Avahi printers

This change extends set of Avahi advertised printers which
works with Gtk's CUPS print backend.

It creates a temporary queue (local printer) for each
Avahi printer in CUPS instead of accessing them directly
(via CUPS library).

This makes some printers work which did not work before and
also gives users more options to change in the print dialog.

This also changes naming of printers to be in accordance with CUPS.
It uses '_' instead of '-' and has hostname appended for CUPS remote
printers.

Merge branch 'matthiasc/for-master' into 'master'

iconbrowser: Make image dnd work again

Closes #3648

See merge request GNOME/gtk!3159

Merge branch 'master' into 'master'

Improve the docs of GtkWidget and GtkGrid

See merge request GNOME/gtk!2946

iconbrowser: Make image dnd work again

We need to drag a texture, not a paintable.

Fixes: #3648

Merge branch 'matthiasc/for-master' into 'master'

docs: Rewrite the long description for GtkDialog

Closes #3646

See merge request GNOME/gtk!3156

Merge branch 'doc-typo' into 'master'

gtkshow: Fix doc typo

See merge request GNOME/gtk!3158

gtkshow: Fix doc typo

docs: Rewrite the long description for GtkDialog

As was pointed out in #3646, some of the content here
was a bit outdated.

Fixes: #3646

Merge branch 'matthiasc/for-master' into 'master'

More work on css transition tests

See merge request GNOME/gtk!3154

Drop the installed test for now

It fails in ci, and I have no idea why.

Merge branch 'pvs-fixes' into 'master'

Pvs fixes

See merge request GNOME/gtk!3155

treemodelfilter: Drop unreachable code

We never get here. The compiler says so.

Pointed out in https://www.viva64.com/en/b/0793/

css: Drop a bit of unreachable code

We never get here. The compiler says so.

Pointed out in https://www.viva64.com/en/b/0793/

a11y: Avoid out-of-bounds access

Don't use the index before we've checked its good.

Pointed out in https://www.viva64.com/en/b/0793/

gdk: Remove a redundant check

We already know desktop_notification_id is not NULL.

Pointed out in https://www.viva64.com/en/b/0793/

messagedialog: Don't initialize twice

We don't need to set these fields more than once.

Pointed out in https://www.viva64.com/en/b/0793/

composetable: Remove a redundant check

We already know seq_index is not NULL here.

Pointed out in https://www.viva64.com/en/b/0793/

x11: A case of argument order confusion

translate_keysym was expecting its arguments the
other way around.

Pointed out in https://www.viva64.com/en/b/0793/

textbtree: Avoid line vs char count confusion

The post_insert_fixup helper function was confused about
its argument order.

Pointed out in https://www.viva64.com/en/b/0793/

listbase: Don't specify the same thing twice

We only need to set EXPLICIT_NOTIFY once.

Pointed out in https://www.viva64.com/en/b/0793/

css: Fix border value parsing

This function was not resetting computed as it meant
to because the last loop was never executed.

Pointed out in https://www.viva64.com/en/b/0793/

vulkan: Fix image uploading by regions

This code did not make sense; it was incrementing
the wrong variable.

Pointed out in https://www.viva64.com/en/b/0793/

docs: Mention css drop-shadow filter

We don't support this filter, currently.

Add more css transition tests

Rename test to transition

Thats what it is about, so name it clearly.

Add missing installed tests too.

Add more css transition tests

Test font size transitions.

testsuite: Pass GDK_DEBUG=default-settings to css tests

Otherwise, settings might creep in and change css defaults.

cssvalue: Cosmetic change

Don't return FALSE from pointer-returning functions.

Improve the css value tests

The test code had some bugs. Fix those, and
print out useful information when tests fail.

css: Fix shadow value equal

This function was not doing the right thing.

Once we are doing the right thing and not compare
shadows as unequal, some reftests that inhibit
snapshots for a few frames now hang forever, since
we are no more redrawing unnecessarily. Fix that
with an explicit queue_draw.

Merge branch 'wip/jimmac/colored-list-image-buttons' into 'master'

Adwaita: allow suggested and destructive action buttons in lists

Closes #3643

See merge request GNOME/gtk!3153

Merge branch 'ebassi/for-master' into 'master'

Graphene is a dependency of Gsk, not Gdk

See merge request GNOME/gtk!3149

Adwaita: allow suggested and destructive action buttons in lists

Fixes https://gitlab.gnome.org/GNOME/gtk/-/issues/3643

listbox: Fix handling of non-focusable rows

This commit modifies the gtk_list_box_row_grab_focus() implementation to
correctly return FALSE in case the row couldn't be focused. This behavior will
also be honored by gtk_list_box_row_focus() to properly handle focus in that
case. To achieve that, the method has been restructured slightly.

Closes #3633

Merge branch 'x11-dnd-fix' into 'master'

x11: Handle X-specific targets in drops

Closes #3642

See merge request GNOME/gtk!3151

x11: Handle X-specific targets in drops

This code is very similar to the handling for these
targets in the clipboard case.

Fixes: #3642

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!3150

Graphene is a dependency of Gsk, not Gdk

So let's put it in the right place when building the introspection data.

Link the data url tests statically

Thats the cleaner way to test internal apis.

Add tests for css value transitions

Just a few cases for now, enough to test a recently
fixed regression with shadow transitions.

Merge branch 'uac.meson.master' into 'master'

gtk4-update-icon-cache: Avoid UAC on 32-bit Windows

Closes #3632

See merge request GNOME/gtk!3141

gtk4-update-icon-cache: Avoid UAC on 32-bit Windows

As the program executable name has 'update' in its filename,
gtk4-update-icon-cache.exe is considered to be an installer program on 32-bit
Windows [1], which will cause the program to fail to run unless it is running
with elevated privileges (i.e. UAC).

Avoid this situation by embedding a manifest file into the final executable
that tells Windows that this is not a program that requires elevation.

Fixes issue #3632.

[1]: https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-vista/cc709628(v=ws.10)?redirectedfrom=MSDN,
     under section "Installer Detection  Technology"

Merge branch 'matthiasc/for-master' into 'master'

css: Allow transitioning different-size shadows

See merge request GNOME/gtk!3145

css: Allow transitioning different-size shadows

The code handles it just fine. The length check was
an erronous addition.

Merge branch 'im-context-work' into 'master'

Some im context work

Closes #1004, #186, and #3521

See merge request GNOME/gtk!3143

composetable: Parse hex escapes too

This was a small omission from the Compose file
syntax that doesn't cost us much to support.

Add a test for this syntax too.

Fixes: #1004

Add tests for string values

Add a test that checks we parse values with
multiple characters correctly.

composetable: Support string values in the cache

Change the cache format to include the character
data that we need to hold string values in the table.

composetable: Don't use GSlice for big blobs

This just doesn't make sense. This will use malloc
anyway, so just call malloc directly.

composetable: Keep multi-char values

Keep string values in the table, and return them
from the check function. This commit temporarily
disables the table caching, since the cache format
does not handle string values yet.

Fixes: #186

Merge branch 'wip/silence-bounds-warnings' into 'master'

gdk/toplevelsize: Remove warnings about exceeding bounds

Closes #3035

See merge request GNOME/gtk!3142

composetable: Parse multi-char values

Rewrite the value parsing function to accept strings
that hold more than a single Unicode character.

composetable: Another step towards multi-char values

Change the parser data structures to hold a string, rather
than a gunichar. We still only put a single Unicode character
into it, currently.

composetable: Prepare for multi character values

Make it possible for gtk_compose_table_check to return
a string instead of just a single Unicode character.
Currently, we only ever return strings holding a single
character, still.

imcontext: Prepare for multi-char values

Reshuffle things so we can easily handle values
that are strings instead of just single Unicode
characters.

composetable: Check algorithmic matching

Just some spot checks, enough to verify the
fix in the previous commit.

composetable: Fix algorithmic matching

The code wasn't paying attention to (lack of) nul-termination
in one place, causing it to not match when it should.

composetable: Add tests for compact table matching

Not very exhaustive, just some spot checks.

imcontext: Code cleanup

Get rid of auxiliary check_table function.

imcontext: Move code around

Move all the checking code to gtkcomposetable.c, and
add api that we can use in tests.

composetable: Add tests for matching

This tests the api we use to match key sequences
against compose tables.

imcontext: Use gtk_compose_table_check

Use the just-introduced api.

composetable: Add api to check tables

This copies the check_table code from gtkimcontextsimple.c,
in order to have an api for tests.